Skip to content

Revert "bazel: update rules_foreign_cc"#17875

Merged
mattklein123 merged 1 commit intoenvoyproxy:mainfrom
keith:revert-17799-ks/revert-revert-bazel-update-rules_foreign_cc-17445-17780
Aug 27, 2021
Merged

Revert "bazel: update rules_foreign_cc"#17875
mattklein123 merged 1 commit intoenvoyproxy:mainfrom
keith:revert-17799-ks/revert-revert-bazel-update-rules_foreign_cc-17445-17780

Conversation

@keith
Copy link
Member

@keith keith commented Aug 26, 2021

Reverts #17799

Seems like this wasn't enough yet again envoyproxy/envoy-mobile#1754

Signed-off-by: Keith Smiley keithbsmiley@gmail.com

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Aug 26, 2021
@repokitteh-read-only
Copy link

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).

🐱

Caused by: #17875 was opened by keith.

see: more, trace.

@phlax
Copy link
Member

phlax commented Aug 27, 2021

@keith you need to fix DCO, not sure if the other errors are genuine

)" (envoyproxy#17780)" (envoyproxy#17799)"

This reverts commit 4b3f4a5.

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
@keith keith force-pushed the revert-17799-ks/revert-revert-bazel-update-rules_foreign_cc-17445-17780 branch from c38dc5f to fd1503c Compare August 27, 2021 17:59
@junr03
Copy link
Member

junr03 commented Aug 27, 2021

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Check envoy-presubmit isn't fully completed, but will still attempt retrying.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #17875 (comment) was created by @junr03.

see: more, trace.

@junr03
Copy link
Member

junr03 commented Aug 27, 2021

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #17875 (comment) was created by @junr03.

see: more, trace.

@mattklein123 mattklein123 merged commit 7760bc0 into envoyproxy:main Aug 27, 2021
@keith keith deleted the revert-17799-ks/revert-revert-bazel-update-rules_foreign_cc-17445-17780 branch August 27, 2021 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deps Approval required for changes to Envoy's external dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants